[UI/#53] TerningBasicDialog #61
Conversation
| Row( | ||
| modifier = Modifier | ||
| .width(60.dp) | ||
| .height(60.dp) |
There was a problem hiding this comment.
aspectRatio 사용해도 좋을 것 같아요!
| modifier = Modifier.padding(bottom = 29.dp), | ||
| verticalArrangement = Arrangement.spacedBy( | ||
| 5.dp, | ||
| Alignment.CenterVertically |
There was a problem hiding this comment.
와우 Arrangement 안에 Alignment 넣을 수 있는거 처음 알았어요..!!
| stringResource(id = R.string.intern_info_d_day) to "2024년 7월 23일", | ||
| stringResource(id = R.string.intern_info_working) to "2개월", | ||
| stringResource(id = R.string.intern_info_start_date) to "2024년 8월" |
There was a problem hiding this comment.
파라미터로 받아오는걸로 수정해야 할 것 같아요!!
There was a problem hiding this comment.
아니 진짜 섬세,, ㅠㅠ 고쳤릅니다!
| } else { | ||
| R.string.dialog_scrap_button | ||
| }, | ||
| onButtonClick = { /*TODO*/ }, |
There was a problem hiding this comment.
이렇게 하면 ripple이 생겨서 noRippleClickable 사용하는게 좋을 것 같아요!
There was a problem hiding this comment.
흠,,근데 제가 noRippleTheme 작성했었는데 적용이 안 되나요,,?
onButtonClick에 아무 로직이 없어서 그런 것 같기도 한데 한번 해보고 ripple 없는 지 확인해주세요!!!
There was a problem hiding this comment.
요거는 유빈이가 만들어준거라서 디폴트로 적용되어 이씁니다!
|
|
||
| when { | ||
| openDialog.value -> { | ||
| TerningBasicDialog( | ||
| onDismissRequest = { openDialog.value = false }, | ||
| content = { | ||
| ScrapDialogContent( | ||
| onDismissRequest = { openDialog.value = false }, | ||
| isScrapped = false | ||
| ) | ||
| } | ||
| ) | ||
| } | ||
| } |
There was a problem hiding this comment.
이거 머지하기 전에 꼭 수정하고 머지해주세요!!
| .background( | ||
| color = White, | ||
| shape = RoundedCornerShape(20.dp) |
There was a problem hiding this comment.
요거 자동으로 White 설정은 안 되는 건가용,,?
There was a problem hiding this comment.
넹 머터리얼 기본 보라색,,.?희미한 그 색이 되더라구욧..
| ) { | ||
| IconButton(onClick = { onDismissRequest() }, modifier = Modifier.padding(6.dp)) { | ||
| Icon( | ||
| painter = painterResource(id = R.drawable.ic_dialog_x_32), |
There was a problem hiding this comment.
줄바꿈!! 해주세요!! 그리구 여기가 X 버튼인 것 같은데 noRippleClickable로 해줘야 할 것 같아요!!
컴포즈는 기본으로 ripple 효과가 들어가기 때문에 요 부분 신경써준다면 감사하겠습니당
| ) | ||
| if (isColorChange) { | ||
| Text(text = "컬러 팔레트") |
| } else { | ||
| R.string.dialog_scrap_button | ||
| }, | ||
| onButtonClick = { /*TODO*/ }, |
There was a problem hiding this comment.
흠,,근데 제가 noRippleTheme 작성했었는데 적용이 안 되나요,,?
onButtonClick에 아무 로직이 없어서 그런 것 같기도 한데 한번 해보고 ripple 없는 지 확인해주세요!!!
| ) { | ||
| Box(modifier = modifier.size(39.dp)) { | ||
| Box(modifier = modifier.wrapContentSize()) { | ||
| Box( |
| ) { | ||
| Box(modifier = modifier.size(39.dp)) { | ||
| Box(modifier = modifier.wrapContentSize()) { | ||
| Box( |
⛳️ Work Description
📸 Screenshot
53.mp4
📢 To Reviewers